Skip to content

fix: count reverts vs features in update-semver skill - #868

Closed
avoidwork wants to merge 1 commit into
mainfrom
fix/update-semver-revert-counting
Closed

fix: count reverts vs features in update-semver skill#868
avoidwork wants to merge 1 commit into
mainfrom
fix/update-semver-revert-counting

Conversation

@avoidwork

Copy link
Copy Markdown
Owner

Description

Corrects the revert-to-feature ratio calculation in the update-semver skill by using the correct git diff range (HEAD1 instead of HEAD2) when counting revert commits vs feature commits.

Type of Change

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactor (no functional changes)
  • Performance improvement
  • CI / build / tooling

Testing

N/A

Coverage

  • Line coverage maintained

Checklist

  • npm run lint passes
  • Tests pass with maintained line coverage
  • No forbidden patterns used
  • Conventional Commit style applied

A Revert "feat: ..." cancels the corresponding feat: commit. If all
features are reverted, treat them as if they never existed — bump
patch instead of minor.
@avoidwork avoidwork added the bug Something isn't working label Aug 25, 2026
@avoidwork avoidwork self-assigned this Aug 25, 2026
@avoidwork avoidwork closed this Aug 25, 2026
@avoidwork
avoidwork deleted the fix/update-semver-revert-counting branch August 25, 2026 00:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant